-
Notifications
You must be signed in to change notification settings - Fork 344
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add %ROM_DIR%, %CFG_DIR%, %INSTALL_DIR% tags #881
base: master
Are you sure you want to change the base?
Conversation
What's the use case for %ES_CFG%? |
Thank you for response.
And the EmulationStation settings have a fixed location (or --home), this can be resolved with a link.
In this situation, by using <command>%ES_CFG%/../the_emulator_A/the_emulator_A.exe %ROM%</command> We could write it using an absolute path, but that would make it less portable. Note @eib has submitted a similar PR #681 in the past, but although I used it as reference, I was unable to find a use for |
This isn't fully clear to me, but let me make a few more comments then. Why not just stick to the home path and call it ES_HOME? The settings folder is meant to be for internal ES usage, it's not immediately clear to me why that is useful to have as a variable rather than home, unless there would be a use case to reading or manipulating the config files for some reason. Thoughts on that? |
Yes As you point out, I don't want to read or manipulate a configuration file.
If we name this |
Ok, but then I am really confused - if your goal is to make things portable, I was assuming you were taking ES with you somewhere in that I am really struggling to understand the use case that would have a portable config file and such - but not ES executable - and that you would need to make something depend on the configuration file folder. I mean, I understand that it works the way you're describing if you choose to implement it that way, but I'd imagine there would be more elegant solutions to accomplish it. Still, since I am not someone who runs this in a portable setting and as such I might be clearly missing something in analyzing the whole use case, I will let others chime in, comment on this and provide guidance for reviewing it. |
@pjft the entire idea is to have the emulator command be 'portable' by relying on some path portion that is calculated at runtime (and then expanded by EmulationStation's launching command code) - without starting ES through a script ( Now, looking at the example give above, a more correct way would be to expand %ES_INSTALL% as the ES install folder and not use the config folder, which can be overrided through %ES_HOME% already. However, it still looks a bit hackish to me. |
Thanks. Correct, I think I am suggesting what you are suggesting - having the variable point at the ES start path (which for me was ES HOME, but I get that it can be a loaded label). I was just struggling with having the variable point at the settings folder instead of the start path, that was my main point of contention. It might have come across incorrectly. |
Hi, there... been a while. ;) I notice my PR #681 and thus the issue of portability (I reported in #680) are still open. (Granted I appear to have merge conflicts.) I see @o-p-a is interested in a shortcut to the ES configuration directory. I was interested in the ES executable directory -- which in code is called "home path", so I referred to it as the "ES Home directory". And we are both interested in shortcuts to the ROM directory. Can I propose a ... merging of ideas?
... the reason I opened #680 was to give the users of EmulationStation (such as myself) the flexibility to use it/organize it our way. P.S. @o-p-a To me, something called "ES_CFG" sounds like the actual config file. If you're looking at the directory containing the config, I think "CFG_DIR" sounds cleaner. Just my thoughts. ;) |
Thank you everyone for quick responses! (and sorry for my late reply) @pjft I apologize for confusion by not telling that I installed EmulationStation to the absolute path As I mentioned above, I couldn't imagine what Based on the discussion, I would like to suggest the following:
...These are the existing tags. In addition to these, how about adding the following:
I think @eib's point about If there are no objections, I'll add a commit along those lines soon. Thanks everyone. |
Add %ROM_DIR%, %ES_CFG% tags in launch commands.
%ROM_DIR%
- Replaced with the dirname (except "/base.ext") of the selected ROM. Use this when you need to chdir to the directory where the ROM.%ES_CFG%
- Replaced with the directory path of the EmulationStation configuration file contains (i.e. your .emulationstation directory). If .emulationstation is symlink, the link will resolved.